spinner: Move G_DEFINE_TYPE() below function definitions

Also remove redundant double definition.

wrapbox: Lookie, an unused variable

spinner: Define desired size at top of file

No magic numbers in the code please

GtkWrapBox: let GtkContainer handle border width

Remove fill options from GtkWrapBox

GtkWidget alignment properties make this unnecessary in new containers.

Remove padding from GtkWrapBox

GtkWidget margins make this unnecessary in new containers.

GtkBox: let GtkContainer handle border width

Add deprecation guards for gtk_widget_get_child_requisition()

demos: Do not use deprecated gtk_widget_get_child_requisition()

Use gtk_size_request_get_size() instead

bgo#486839 - Fix path bar size group in the file chooser

Since the location button is hidden in save mode, we need to add the
path bar to the size group too. The location button still has to be in
the group though, because it's larger than the path bar (when
shown). Instead of using the recent/search icons, add their hboxes so
that themed widget spacings don't introduce variations.

Use gtk_size_request_get_size() instead deprecated gtk_widget_get_child_requisition()

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=629177

Fixed GtkWrapBox to allocate variable row AND column heights when in ALIGNED mode

With this new approach at request and allocate time, the average child size
is used to determine a good guess at how many columns will fit the box
width; afterwards extra columns are appended and checked to fit.
Then the row heights are calculated based on height-for-width of each
child in the row which now may have individual widths.

Make testwrapbox scrolled window scrollbars automatic.

Did this in the hope to reproduce infinite recursion bugs
with height-for-width in scrolled windows (see complex bgo #611740
for reference for now).

Changed GtkWrapBox code to avoid a growing GArray of a known size.

[GdkWindowCache] Don't ignore the CompositeOverlayWindow

Add the composite overlay window to the cache, as this can be a reasonable Xdnd proxy as well.

This is only done when the screen is composited in order to avoid mapping
the COW. We assume that the CM is using the COW (which is true for pretty
much any CM currently in use).

https://bugzilla.gnome.org/show_bug.cgi?id=601731

Update gtkstyle.c to account for removal of row-ending-details

The detail strings now have more "detail" by default, so gtkstyle.c
needed to be updated to properly handle this.  Tests like testtreeview,
testtreesort now have proper background drawing again.

This strncmp trick was the best I could think of so quickly, if anybody
has an idea to do this in a better way, let me know.

Mark declaration of symbol _gtk_text_unknown_char_utf8 as extern

So that the symbol is not re-created each time the header is included.
The symbol is defined in gtktexttypes.c already.

Use accessor functions to access GtkWidget

Update for GDK key name changes

Update Quartz backend for GDK key name changes

Updated Japanese translation.

Add padding and alignment tests to testadjustsize.c

Add margin and alignment properties to GtkWidget

h-align = START,END,CENTER,FILL
v-align = START,END,CENTER,FILL
margin-left,right,top,bottom
margin

These should obsolete all such similar properties on
layout containers, GtkMisc, GtkAlignment, GtkContainer::border-width

Margin is outside the size request.
If margin were not outside the set_size_request() it would not work the
same way as container-supplied (child property) padding.

Conceptually set_size_request() forces the value from the subclass
(the original unadjusted request) and then we go on to adjust
the request further by adding the margin.

GtkButton: let GtkContainer handle border width

add gtk_container_class_handle_border_width() so subclasses can ignore border_width

A subclass calls gtk_container_class_handle_border_width()
in its class_init

This marks the subclass as expecting GtkContainer to deal with
border width automatically, which GtkContainer then does.

Add testadjustsize test, to test new adjust size methods and related features

This will test size adjust, and interactions with other padding and border

GtkWidget: add adjust_size_request adjust_size_allocation virtual funcs

Use these new methods to handle set_size_request (aka aux_info)
inside gtkwidget.c, instead of having external code mess with it.

The virtual functions can be used for other purposes in the
future. For example, GtkContainer::border_width could be
automatically implemented for all container subclasses.

Use _gtk_widget_get_aux_info_or_defaults() when possible in gtkwidget.c

Did not update uses in other files because the plan is to
get rid of those other uses anyhow. So don't want to make
this function available in the header.

add _gtk_widget_get_aux_info_or_defaults()

This is better than peeking aux info then testing != NULL
in several ways:
- it returns const aux info so if we don't create we can't write
- it ensures that the default we assume if aux_info is NULL is
  the same as the default we set if we've created the aux info
- it avoids typing in != NULL checks

Fix more SizeRequest implementations to avoid recursive calls to wrapper API

GtkFrame, GtkComboBox, GtkExpander, GtkMenu, GtkWrapBox

These are all the examples I could find so far.

https://bugzilla.gnome.org/show_bug.cgi?id=628829

Warn about recursively calling size req wrappers on the same object and orientation

We are not re-entrant and there is no reason for widgets to
do this, most likely they'll just get unexpected bugs because
the wrappers may modify the request.

Computing the request should logically rely only on the
widget itself, not on any adjustments caused by set_size_request,
size groups, and so forth.

https://bugzilla.gnome.org/show_bug.cgi?id=628829

default impls of width_for_height,hfw should chain directly not use wrapper API

In GtkBin and GtkWidget we tried to provide handy defaults that
call get_width if there's no get_width_for_height and
get_height for get_height_for_width.

However, they used the wrapper API on GtkSizeRequest instead of
chaining directly to the other method implementation.

This could result in all kinds of surprising behavior, for example,
get_width_for_height() would now already include the effects of set_size_request().

If nothing else it's inefficient. But it's just conceptually wrong,
because to chain to another implementation, we should call the other
implementation, not call a wrapper around the other implementation
(when we're already inside a previous invocation of the wrapper,
i.e. compute_size_for_orientation() ends up reinvoking itself
in the same orientation on the same object which it pretty
likely isn't intending to do)

https://bugzilla.gnome.org/show_bug.cgi?id=628829

Use gint16 for GtkBorder

32K of border ought to be enough for any pixel dimensions. At least
until screens are so huge we start using doubles.

This saves a nice 64 bits of space when we have a GtkBorder
stored somewhere.

Signed integers are used to avoid surprising unsigned math
issues. Just search GTK's whole git log from inception
for "unsigned" if you want to find any number of commits
fixing signed/unsigned bugs.

https://bugzilla.gnome.org/show_bug.cgi?id=629387

Fix some compilation warnings

GScanner token field is declared as GTokenType instead a guint
See Glib bug: https://bugzilla.gnome.org/show_bug.cgi?id=627890

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=627962

docs: Fix gdk_display_get_core_pointer() docs

Point to gdk_device_manager_get_client_pointer(),
not gdk_display_get_client_pointer().

style: Fix drawing of insensitive text

When writing the original code I erroneously assumed that the current
point of the cairo context would be saved by cairo_save/restore(), but
of course the current point is part of the path and therefor isn't
saved.

Also do a cairo_new_path() before rendering any text so that we are sure
the text ends up at the right spot.

Don't use g_idle_add to schedule idles in GDK

We need to use gdk_threads_add_idle, in order to keep GDK code
under the GDK lock.

Bug 629277

gtk/gtkwidget.c: Use accessor functions to access GtkWindowGroup

Revert "gtk/gtkmain.c: Use accessor functions to access GtkWindowGroup"

This reverts commit 65b9fc3e5bbae5b5efb4568c02df60939956f722.

gtk/gtkmain.c: Use accessor functions to access GtkWindowGroup

GtkWindow: move public members to private structure

Signed-off-by: Christian Dywan <christian@twotoasts.de>

Bump version to 2.90.8

Include gdkkeysyms.h in gdk.h

Followup to bug 629093

Gail: Remove gail_cell_type_add_action_interface, as it no longer does anything

Gail: Take fill/page size into account when returning a range's maximum value

Updated Norwegian bokmål translation

Some more updates

Update potfiles

Really stop exporting text-related variables

Remove GDK_DISPLAY() usage

Update NEWS for 2.90.7

Drop the gdk_display variable from gdk

This was the last exported variable; it wasn't multihead safe,
and there's easy replacement with gdk_display_get_default().
Also drop the GDK_DISPLAY() macro which was just a wrapper around
the variable.

Fix the docs build after GtkItem removal

Adding note to docs of gtk_widget_set_window()

gtk_widget_set_window() does not add a reference to the
owned window... this is an exception to most gtk_foo_set_bar()
functions where as it acts as if it were named gtk_widget_take_window().

Adding note to the docs to avoid people giving the window to the
widget and naturally proceeding to unref the window.

widget: Fix SEGV when no focus widget exists

gail: Add missing include

Avoid potential DLL hijacking in Wintab code

Load Wintab32.dll from an absolute path. A proper Wintab32.dll, if
present, should be in the Windows system directory, so load it from
there.

Use accessor functions

Fix gail build

Adapt to the GDK_KEY changes

Fixed GtkTextView & GtkIconView to consult it's true previous size request

Fixed issues in my previous patch for bug 626939 removing GtkRequisition
cache: these widgets monitor the previous requested size and decide whether
to queue a resize when the content changes based on it's prior request.

Moved GtkSizeRequest cache to GtkWidget->priv

Now that we have a private data installed directly on
the GtkWidget instance it makes no sense to cache the size
requests on widget qdata. This change will generally make
GTK+ memory less fragmented as well as significantly speed
up the size request process.

Use accessor functions in Win32-only code

Don't use the same name for a function that used to be a variable

Rename the gtk_major_version() etc functions I introduced yesterday to
start with gtk_get. Avoids misleading results in client programs whose
developers that don't notice the change or the compiler warnings, and
when recompiling against gtk3 then use the function addresses as the
version numbers.

Fix build of gail after GtkItem removal

 Remove GtkItem

With the demise of GtkList and GtkTree, it has GtkMenuItem as sole
derived class, and is not really adding any value as a separate class.
Its few useful features have been merged into GtkMenuItem.

Bug 629104

gtk/gtkhandlebox.c: Do not use gdk_window_get_deskrelative_origin()

This completes commit bd277fad50704f6a39619cb418204bdfee112b98

porting guide: Update to mention GDK_KEY_ transition

GDK: Prefix key names with KEY_

The keysyms create a lot of potential namespace conflicts for
C, and are especially problematic for introspection, where we take
constants into the namespace, so GDK_Display conflicts with GdkDisplay.

For C application compatiblity, add gdkkeysyms-compat.h which uses
the old names.

Just one user in GTK+ continues to use gdkkeysyms-compat.h, which is
the gtkimcontextsimple.c, since porting that requires porting more
custom Perl code.

gdk: Update gdkkeysyms.h

In preparation for adding a namespace prefix for introspection.

Remove deprecated gdk_window_get_deskrelative_origin() and its implementations

Drop GDK_DEBUG_GC and _IMAGE as GDK doesn't do that any more

Remove a duplicate variable

Bug#629110: gtk_spinner_accessible_get_type always returns null

Use G_CALLBACK() instead deprecated GTK_SIGNAL_FUNC()

docs: Remove GtkFileSelection deprecated docs

GtkWidget: Move public members to private structure

gtk/gtknotebook.c: use accessor functions to access GtkWidget

gtk/gtkarrow.c: use accessor functions to access GtkWidget

gtk/gtkpathbar.c: use accessor functions to access GtkWidget

gtk/gtkruler.c: use accessor functions to access GtkWidget

gtk/gtkseparator.c: use accessor functions to access GtkWidget

gtk/gtkhandlebox.c: use accessor functions to access GtkWidget

gtk/gtkiconview.c: use accessor functions to access GtkWidget

gtk/gtkimage.c: use accessor functions to access GtkWidget

gtk/gtkmenu.c: use accessor functions to access GtkWidget

gtk/gtkmisc.c: use accessor functions to access GtkWidget

gtk/gtkspinbutton.c: use accessor functions to access GtkWidget

gtk/gtktextview.c: use accessor functions to access GtkWidget

gtk/gtktreeview.c: use accessor functions to access GtkWidget

gtk/gtkwindow.c: Fix warning

Cast to GtkSizeRequest

This was added in
commit b654afcb480c4f971436bd82a7b20f86686a9c02

Get rid of the rest of variables exported from libgtk

Especially the gtk_*_type ones in gtktexttypes.h were mentioned in
gtk.symbols presumably by accident. That header isn't even installed,
so no way can they be supposed to be public.

gtk_text_attr_appearance_type is from the installed but "semi-private"
gtktextlayout.h, so drop that one too from gtk.symbols for now.

The use of gtk_text_unknown_char_utf8 is bit of a mess. Code in a few
files knew implicitly that it is three bytes. Define a symbolic name
for the length of it instead. Add an exported function
gtk_text_unknown_char_utf8_gtk_tests_only() that returns a pointer to
it just for the sake of gtk/tests/textbuffer.c. Prefix the variable
with an underscore.

I doubt the usefulness of the test_utf8() in textbuffer.c. If it could
be dropped, gtk_text_unknown_char_utf8_gtk_tests_only() could be
dropped, too.

Replace gtk_debug_flags with getter and setter functions

Preferrably should be made just into a local variable for libgtk like
_gdk_debug_flags for libgdk. But for now used by
gtk/tests/textbuffer.c and modules/printbackends/cups/gtkprintbackendcups.c.

Remove also the docs for the gdk_threads_mutex variable

Turn the gtk version and age variables into functions

Having variables exported from a DLL is slightly painful and
potentially error-prone on Windows, so let's try get rid of them now
when we can. Starting with these.

gtk/gtkwindow.c: Use gtk_size_request_get_size() instead gtk_widget_get_requisition()

Completely removed requisition cache from GtkWidget instance structure.

Since we have a new mechanism for requesting sizes: GtkSizeRequestIface;
it makes no sense to maintain this cache on the GtkWidget structure...
removing the requisition cache however does not break the old "size-request"
signal which is there for backwards compatability reasons.

In any case widget->requisition should not have been accessed,
gtk_widget_get_child_requisition() would have been the correct way
to consult the cache.

This commit also deprecates the newly added gtk_widget_get_requisition()
API and makes it fallback on gtk_size_request_get_size().

tests: Use gtk_widget_set_size_request() instead gtk_widget_set_usize()

gtknotebook: Do not mention deprecated API

demos: Do not use deprecated API

gtksocket: Remove mention to gtk_socket_steal() deprecated function